The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
docs	Ready	Preview, Comment	Apr 14, 2026 10:43pm

PR Summary

Low Risk
Low risk UI-only change that tweaks drag/drop event handling and requestAnimationFrame cleanup; main risk is unintended drag state resets if events fire unexpectedly.

Overview
Prevents the workflow sidebar from continuing to auto-scroll after the user drags outside the scrollable container.

This adds dragleave/global drop+dragend listeners during an active drag to cancel the auto-scroll animation and clear hover/drop indicators, and ensures auto-scroll can be restarted if dragover resumes after being cancelled.

^{Reviewed by Cursor Bugbot for commit 6d5cad0. Bugbot is set up for automated code reviews on this repo. Configure here.}

Greptile Summary

This PR adds a useEffect to the useDragDrop hook that listens for native dragleave events on the scroll container and calls handleDragEnd() when the drag cursor exits the container boundary (i.e., relatedTarget is null or outside the container). This cleanly stops the auto-scroll loop that was previously left running whenever a file drag passed over the sidebar and then exited.

Confidence Score: 5/5

Safe to merge — the fix is minimal, correctly scoped, and idempotent.

The new useEffect is correctly structured: it only attaches when isDragging is true, uses the container.contains(relatedTarget) guard to avoid false positives from intra-container drag events, and the cleanup reliably removes the listener. handleDragEnd is idempotent so duplicate calls are harmless. All remaining observations are P2 or below.

No files require special attention.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant User
    participant Browser
    participant ScrollContainer
    participant useDragDrop

    User->>Browser: Drag file/item over sidebar
    Browser->>ScrollContainer: dragover event
    ScrollContainer->>useDragDrop: initDragOver() → isDragging = true
    useDragDrop->>useDragDrop: useEffect attaches dragleave listener

    loop Auto-scroll loop (while isDragging)
        useDragDrop->>ScrollContainer: requestAnimationFrame(handleAutoScroll)
    end

    User->>Browser: Drag cursor leaves container boundary
    Browser->>ScrollContainer: dragleave (relatedTarget = null or outside)
    ScrollContainer->>useDragDrop: onLeave() → !container.contains(related)
    useDragDrop->>useDragDrop: handleDragEnd() → isDragging = false
    useDragDrop->>useDragDrop: useEffect cleanup removes dragleave listener
    useDragDrop->>ScrollContainer: cancelAnimationFrame (auto-scroll stops)

_{Reviews (1): Last reviewed commit: "fix(ui): stop scrolling on leaving workf..." | Re-trigger Greptile}